Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move providers to new namespace #604

Merged
merged 11 commits into from
May 13, 2017
Merged

Conversation

Nyholm
Copy link
Member

@Nyholm Nyholm commented Apr 28, 2017

This is a first PR for #578

I've just done the Chain provider the moment. All other providers will follow the same pattern.

TODO

  • Move the rest of the providers
  • Update change log
  • Make sure travis runs the tests

@Nyholm Nyholm added this to the 4.0.0 milestone Apr 28, 2017
@willdurand willdurand self-requested a review April 28, 2017 16:19
Nyholm added 10 commits May 12, 2017 21:30
* fix

* bugfix

* Make sure we run tests for common

* tests

* debug

* Added some config

* Updated path to phpunit

* debug

* debub

* added phpunit file

* minor

* Fixed to get phpunit-bridge to work

* Do not use simple-phpunit

* Fixed broken tests

* Fixed broken tests

* Use curl-client and guzzlehttp/psr7: ^1.4

* Remove Guzzle

* Require latest version of CurlClient
* Make sure tests run

* Use Curl

* Renamed cached files
@Nyholm
Copy link
Member Author

Nyholm commented May 13, 2017

Im done with this PR. Deps low will fail because it is using code in master.

@Nyholm Nyholm merged commit 3180c7b into geocoder-php:master May 13, 2017
@Nyholm Nyholm deleted the namespace branch May 13, 2017 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant